Make cpu param to continue_hypercall_on_cpu() an unsigned integer.

Negative input makes no sense, and this makes the input range check
correct.

Signed-off-by: Keir Fraser <keir.fraser@citrix.com>

diff --git a/xen/common/domain.c b/xen/common/domain.c
index 6688b158d940d1db1b3ddbcca565fcfd09f54098..6707b332464b1c1026e475eef3472b10d57bce4d 100644 (file)
--- a/xen/common/domain.c
+++ b/xen/common/domain.c
@@ -943,7 +943,8 @@ static void continue_hypercall_tasklet_handler(unsigned long _info)
     }
 }
 
-int continue_hypercall_on_cpu(int cpu, long (*func)(void *data), void *data)
+int continue_hypercall_on_cpu(
+    unsigned int cpu, long (*func)(void *data), void *data)
 {
     struct migrate_info *info;
 

diff --git a/xen/include/xen/domain.h b/xen/include/xen/domain.h
index 3699291aca7163518669c2081a6ea82dbc3fbd7e..79711bdb29e97d0e3991a45d666aee9154287d8a 100644 (file)
--- a/xen/include/xen/domain.h
+++ b/xen/include/xen/domain.h
@@ -64,7 +64,8 @@ bool_t domctl_lock_acquire(void);
 void domctl_lock_release(void);
 
 /* Continue the current hypercall via func(data) on specified cpu. */
-int continue_hypercall_on_cpu(int cpu, long (*func)(void *data), void *data);
+int continue_hypercall_on_cpu(
+    unsigned int cpu, long (*func)(void *data), void *data);
 
 extern unsigned int xen_processor_pmbits;